Skip to content

[17.0][ADD] report_printed_flag#1161

Draft
atgalvez08 wants to merge 1 commit into
OCA:17.0from
BinhexTeam:17.0-add-report_printed_flag
Draft

[17.0][ADD] report_printed_flag#1161
atgalvez08 wants to merge 1 commit into
OCA:17.0from
BinhexTeam:17.0-add-report_printed_flag

Conversation

@atgalvez08
Copy link
Copy Markdown

@BinhexTeam

This module provides a reusable framework to track printed QWeb PDF
reports across business models.

It introduces configurable printed tracking with optional logging
and reusable integration through a generic mixin.

Features

  • Configurable printed tracking per model and report
  • Optional printed logs
  • Optional printed report names
  • Multi-company support
  • Reusable mixin for business models
  • Integration with Odoo report engine

Included components

  • report.printed.config
  • report.printed.config.line
  • report.printed.log
  • report.printed.mixin

Design considerations

The implementation is intentionally generic and reusable.

The module:

  • avoids model-specific assumptions
  • isolates report integration logic
  • uses a reusable abstract mixin
  • supports extension through satellite modules

Technical details

  • Hooks into ir.actions.report
  • Supports QWeb PDF reports (qweb-pdf)
  • Uses Many2oneReference for generic log linkage
  • Handles optional logging and report name computation

Future extensions

This module is designed to be extended by modules such as:

  • report_printed_flag_account
  • report_printed_flag_stock

Dependencies

  • base
  • web

Testing

Includes automated tests covering:

  • printed flag assignment
  • printed logs creation
  • report configuration filtering
  • multi-company behavior
  • models without printed support
  • printed report names computation

Checklist

  • Module follows OCA structure
  • README aligned with implementation
  • Tests included
  • No breaking changes
  • License headers included

Copy link
Copy Markdown
Contributor

@rrebollo rrebollo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please consider my suggestions. I think you'll find some of them compelling.

Comment thread report_printed_flag/models/ir_actions_report.py Outdated
Comment thread report_printed_flag/models/ir_actions_report.py Outdated
Comment thread report_printed_flag/models/ir_actions_report.py Outdated
Comment thread report_printed_flag/models/ir_actions_report.py Outdated
Comment thread report_printed_flag/models/ir_actions_report.py Outdated
Comment thread report_printed_flag/views/report_printed_config_views.xml Outdated
Comment thread report_printed_flag/views/report_printed_log_views.xml Outdated
Comment thread report_printed_flag/__manifest__.py Outdated
Comment thread report_printed_flag/__manifest__.py Outdated
Comment thread report_printed_flag/__manifest__.py
@atgalvez08 atgalvez08 force-pushed the 17.0-add-report_printed_flag branch from 3b6049d to c58eed7 Compare May 14, 2026 10:57
…ramework

This module provides a reusable framework to track printed QWeb PDF
reports across business models.

It introduces:
- printed configuration models
- printed logs
- reusable printed mixin
- report integration hooks

Features:
- configurable printed tracking per model and report
- optional printed logs
- optional printed report names
- multi-company support
- reusable mixin for extension modules

The implementation is designed to be extended by satellite modules
such as account and stock integrations.
@atgalvez08 atgalvez08 force-pushed the 17.0-add-report_printed_flag branch from c58eed7 to d751db0 Compare May 14, 2026 11:17
@atgalvez08 atgalvez08 requested a review from rrebollo May 14, 2026 11:42
Copy link
Copy Markdown
Contributor

@rrebollo rrebollo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review. LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:report_printed_flag Module report_printed_flag series:17.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants